Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Allow passing pull request number #20

Merged
merged 4 commits into from
Sep 14, 2020
Merged

Allow passing pull request number #20

merged 4 commits into from
Sep 14, 2020

Conversation

Siemian
Copy link
Contributor

@Siemian Siemian commented Sep 11, 2020

In our workflow tests are triggered by push which means pull request payload is not available. This PR introduce possibility to pass PR number value directly to the action.

@hannseman
Copy link
Contributor

Many thanks for your PR, looks good! Unfortunately we got some conflicts, would you mind resolving them? Thanks!

# Conflicts:
#	README.md
#	src/action.js
#	src/action.test.js
@Siemian
Copy link
Contributor Author

Siemian commented Sep 14, 2020

@hannseman no worries, fixed already 😉

@hannseman
Copy link
Contributor

Super, thanks!

@hannseman hannseman merged commit b3d1f48 into 5monkeys:master Sep 14, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants